-
Notifications
You must be signed in to change notification settings - Fork 208
(android) added bannerHeight to banner LOAD event (#361) #373
base: master
Are you sure you want to change the base?
Conversation
|
Hi, any progress on this? |
|
please merge this fix!! it's extremely helpful! |
|
please merge.. I need it |
|
What does it help with? |
|
having the height in the LOAD event is very useful since the height varies, it saves doing unnecessary calculations for the layout after the the banner is loaded |
|
|
||
| data.put("bannerHeight", heightDp); | ||
| } catch (JSONException e) { | ||
| e.printStackTrace(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice if this propagates to the webview console too.
|
Hi, needed also here. I have a jquery cordova app that needs to be properly resized and if banner is shown I loose bottom buttons... |
|
Any progress on this? |
|
Just noticed if I test on a device running API 30 or API R the banner height that gets returned is actually the device height (so in this case 14400) - is it just me or are others seeing the same issue? Any ideas?
|
Same change implemented on android